Skip to content

Feature/gh32 implement sc tag#33

Merged
BenjiMilan merged 2 commits intodevelopfrom
feature/gh32_implement_sc_tag
Feb 28, 2026
Merged

Feature/gh32 implement sc tag#33
BenjiMilan merged 2 commits intodevelopfrom
feature/gh32_implement_sc_tag

Conversation

@BenjiMilan
Copy link
Contributor

@BenjiMilan BenjiMilan commented Feb 25, 2026

Closes #32

Implement sc tag:

tag check
tag create
tag list
tag push
tag rm
tag show

@BenjiMilan BenjiMilan requested a review from TB-1993 February 25, 2026 13:37
@BenjiMilan BenjiMilan self-assigned this Feb 25, 2026
Copilot AI review requested due to automatic review settings February 25, 2026 13:37
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Implements a new sc tag CLI group to manage git tags across either a single git repo or a repo-style multi-project workspace, wiring the CLI into SCBranching and adding the underlying tag command implementations.

Changes:

  • Adds sc tag subcommands: list, show, create, rm, push, check.
  • Introduces Tag* command implementations to operate on manifest + projects (repo mode) or the current repo (git mode).
  • Renames CLI calls from SCBranching.sc_* to SCBranching.* for clean/status/reset.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 7 comments.

File Description
src/sc/branching_cli.py Adds tag command group and subcommands; updates clean/status/reset to new method names.
src/sc/branching/commands/tag.py New implementations for tag list/show/create/rm/push/check across repo or git project types.
src/sc/branching/branching.py Wires new tag commands into SCBranching and exposes static methods for CLI use.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

@TB-1993 TB-1993 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this looks good. I think we should get a future upgrade ticket for copilots suggestions so we can capture the errors and suggest to users how to fix them. But for now this is good. Ship it!

@BenjiMilan BenjiMilan merged commit 35b4501 into develop Feb 28, 2026
9 checks passed
@BenjiMilan BenjiMilan deleted the feature/gh32_implement_sc_tag branch February 28, 2026 08:54
@github-actions github-actions bot locked and limited conversation to collaborators Feb 28, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature: Add sc tag functionality

3 participants